Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docs: CRD reference topic edits #2453

Merged
merged 2 commits into from
Nov 1, 2022
Merged

Conversation

karengermond
Copy link
Contributor

@karengermond karengermond commented Oct 31, 2022

PR Description

Edited

agent/operator/apis/monitoring/v1alpha1/
deployment.go
doc.go
group.go
types_integrations.go
types_logs.go

Will finish editing topics in this issue: #2452

Which issue(s) this PR fixes

Fixes #2451

Notes to the Reviewer

PR Checklist

  • CHANGELOG updated
  • Documentation added
  • Tests updated

@karengermond karengermond linked an issue Oct 31, 2022 that may be closed by this pull request
@karengermond karengermond changed the title Edited some of the CRD reference topics CRD reference topic edits Oct 31, 2022
@karengermond karengermond changed the title CRD reference topic edits Docs: CRD reference topic edits Oct 31, 2022
Use the following to run the script in a container:

```
docker build -t grafana/agent-build-image:0.18.0 ./build-image
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This version number is too specific, maybe we could just use latests as a tag.

Copy link
Contributor

@marctc marctc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Left one minor comment, but the rest LGTM

Use the following to run the script in a container:

```
docker build -t grafana/agent-build-image:0.18.0 ./build-image
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
docker build -t grafana/agent-build-image:0.18.0 ./build-image
docker build -t grafana/agent-build-image:latest ./build-image


```
docker build -t grafana/agent-build-image:0.18.0 ./build-image
BUILD_IMAGE=grafana/agent-build-image:0.18.0 USE_CONTAINER=1 make generate-crds
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
BUILD_IMAGE=grafana/agent-build-image:0.18.0 USE_CONTAINER=1 make generate-crds
BUILD_IMAGE=grafana/agent-build-image:latest USE_CONTAINER=1 make generate-crds

@karengermond karengermond enabled auto-merge (squash) November 1, 2022 10:48
@karengermond karengermond merged commit b469f85 into main Nov 1, 2022
@karengermond karengermond deleted the 2451-docs-edit-and-build-api-docs branch November 1, 2022 10:53
@github-actions github-actions bot added the frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed. label Mar 16, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Docs: Edit and build Agent Operator CRD reference docs
2 participants